Skip to content

Add a stop job button that allows users to stop jobs#21921

Merged
ahmedhamidawan merged 9 commits intogalaxyproject:devfrom
ahmedhamidawan:add_stop_job_button
May 5, 2026
Merged

Add a stop job button that allows users to stop jobs#21921
ahmedhamidawan merged 9 commits intogalaxyproject:devfrom
ahmedhamidawan:add_stop_job_button

Conversation

@ahmedhamidawan
Copy link
Copy Markdown
Member

@ahmedhamidawan ahmedhamidawan commented Feb 25, 2026

Uses the jobs DELETE endpoint to cancel the job.

firefox_Y8KwRWJOE6

Is a step in the direction of fixing #21911

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

…mation`

Also removes the existing `JobState` component rendered in the `JobStepJobs` modal header.

Without this, when in the `JobStepJobs` component in the invocation view, we were not able to stop a job shown in the table (it would open the job modal instead).
Uses Galaxy base component buttons, indicates when on a route, uses `to` props etc.
@ahmedhamidawan ahmedhamidawan marked this pull request as ready for review April 22, 2026 23:24
Copy link
Copy Markdown
Contributor

@davelopez davelopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great to me! Thank you!

@ahmedhamidawan ahmedhamidawan merged commit 058702d into galaxyproject:dev May 5, 2026
33 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in Galaxy Dev - weeklies May 5, 2026
@ahmedhamidawan ahmedhamidawan deleted the add_stop_job_button branch May 5, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

2 participants